Update list_balances documentation#421
Conversation
|
|
||
| /// Retrieves an overview of all known balances. | ||
| /// | ||
| /// Note that to retrieve the latest on-chain balance, you need to call [`Node::sync_wallets`] first. |
There was a problem hiding this comment.
No, actually use of sync_wallets is more or less discouraged. Users should use the background syncing whenever possible.
There was a problem hiding this comment.
Noted. Do you think we should document that in the README? With a local experiment, sync_wallet was the only way I was able to get it to update the on-chain balance.
There was a problem hiding this comment.
Noted. Do you think we should document that in the README?
Mhh, so it is already noted on the sync_wallet docs. I guess that should be sufficient? It's surely not so important that it needs to go into the README?
With a local experiment, sync_wallet was the only way I was able to get it to update the on-chain balance.
Huh, I'd be interested to learn more about that. You should get the update eventually, on Esplora the default onchain_wallet_sync_interval_secs is 80 seconds to avoid hitting rate-limiting. So you should see updates after that the latest, or can of course try to reduce the interval as long as you don't see a lot of HTTP 429 errors popping up.
2 participants
No description provided.